[PATCH] D155617: Pass to annotate functions with appropriate optimization level.

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 7 09:59:17 PDT 2023


mtrofin added inline comments.


================
Comment at: llvm/test/Transforms/ForcedFunctionAttrs/FunctionAnnotation.csv:1
+FunctionName,OptimizationLevel
+first_function,opt-level=O1
----------------
Puneeth-A-R wrote:
> aeubanks wrote:
> > remove?
> @mtrofin? We had decided upon having the header, should I go ahead & remove it?
I'm ambivalent, I kind of like it's there because it's easy to read (in the test, esp.), but if @aeubanks really wants it out, it's also easy to remove.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155617/new/

https://reviews.llvm.org/D155617



More information about the llvm-commits mailing list