[flang] Fix preprocessor regression (#134405)

For numeric kind suffixes like 1_foo, the preprocessor should be able to
perform macro replacement for macros named either "_foo" or "foo".

Fixes https://github.com/llvm/llvm-project/issues/133399.
This commit is contained in:
Peter Klausler 2025-04-09 12:30:05 -07:00 committed by GitHub
parent b92f4d4c33
commit be133ff4c4
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 21 additions and 5 deletions

View File

@ -892,11 +892,24 @@ bool Prescanner::HandleExponent(TokenSequence &tokens) {
}
bool Prescanner::HandleKindSuffix(TokenSequence &tokens) {
if (*at_ == '_' && IsLegalInIdentifier(*SkipWhiteSpace(at_ + 1))) {
EmitCharAndAdvance(tokens, *at_);
if (IsLegalIdentifierStart(*at_)) {
// The kind specifier might be a macro, so put it into its own token.
if (*at_ == '_' && IsLegalInIdentifier(at_[1])) {
// The kind specifier might be a macro (with or without its leading
// underscore); put it into its own token if it has been defined.
const char *p{at_ + 1};
while (IsLegalInIdentifier(*++p)) {
}
if (CharBlock id{at_, static_cast<std::size_t>(p - at_)};
preprocessor_.IsNameDefined(id)) {
// In 1.0e0_foo, "_foo" is a defined name; retain the
// underscore
tokens.CloseToken();
} else {
EmitCharAndAdvance(tokens, '_');
if (CharBlock id{at_, static_cast<std::size_t>(p - at_)};
preprocessor_.IsNameDefined(id)) {
// In 1.0e0_foo, "foo" is a defined name
tokens.CloseToken();
}
}
while (IsLegalInIdentifier(*at_)) {
EmitCharAndAdvance(tokens, *at_);

View File

@ -187,7 +187,7 @@ TokenSequence &TokenSequence::ToLowerCase() {
} else if (*p == 'h' || *p == 'H') {
// Hollerith
*p = 'h';
} else if (*p == '_') {
} else if (*p == '_' && p + 1 < limit && (p[1] == '"' || p[1] == '\'')) {
// kind-prefixed character literal (e.g., 1_"ABC")
} else {
// exponent

View File

@ -1,6 +1,9 @@
! RUN: %flang -E %s 2>&1 | FileCheck %s
#define n k
#define _m _p
parameter(n=4)
!CHECK: print *,1_k
print *,1_n
!CHECK: print *,1_p
print *,1_m
end