unterumarmung added inline comments.
Comment at: clang-tools-extra/clang-tidy/performance/EnumSizeCheck.cpp:127-128
+
+ diag(MatchedDecl->getLocation(), "enum %0 derive from %1 of size %2 bytes, "
+ "derive from '%3' to reduce it size to %4")
+
unterumarmung added inline comments.
Comment at: clang/include/clang/Driver/Options.td:4131
def : Flag<["-"], "no-integrated-as">, Alias,
- Flags<[CC1Option, NoXarchOption]>;
+ Flags<[CC1Option,FlangOption,NoXarchOption]>;
rovka wrote:
> awarzynski
unterumarmung added inline comments.
Comment at: clang/include/clang/Driver/Options.td:4131
def : Flag<["-"], "no-integrated-as">, Alias,
- Flags<[CC1Option, NoXarchOption]>;
+ Flags<[CC1Option,FlangOption,NoXarchOption]>;
Why not to add `FC1Option`
unterumarmung added a comment.
@ekieri, hello! Thank you for the comment, it is really helpful!
It turns out that I've researched the issue poorly.
Actually, `nvfortran` compiler forbids to use `INTENT` attribute on local
variables, even though a "mod" file can be generated. It generates a
unterumarmung updated this revision to Diff 420437.
unterumarmung added a comment.
Fixed commit message
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123113/new/
https://reviews.llvm.org/D123113
Files:
clang/include/clang/Driver/Options.td
unterumarmung updated this revision to Diff 420436.
unterumarmung retitled this revision from "[Flang] Add `INTENT` for non-dummy
arguments extension nvfortran compiler supports using INTENT attributes for
non-dummy arguments. For example, for local variables." to "[Flang] Add
`INTENT` for
unterumarmung created this revision.
Herald added a reviewer: sscalpone.
Herald added projects: Flang, All.
unterumarmung requested review of this revision.
Herald added subscribers: cfe-commits, sstefan1, jdoerfert, MaskRay.
Herald added a reviewer: jdoerfert.
Herald added a project: clang.
This
