[all-commits] [llvm/llvm-project] 2f6726: [flang] Switch from %f18 to %flang_fc1 in a test

Andrzej WarzyƄski via All-commits all-commits at lists.llvm.org
Fri Apr 23 08:10:42 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2f67267a93c87261414a4aa4c6cb9d20a489a0df
      https://github.com/llvm/llvm-project/commit/2f67267a93c87261414a4aa4c6cb9d20a489a0df
  Author: Andrzej Warzynski <andrzej.warzynski at arm.com>
  Date:   2021-04-23 (Fri, 23 Apr 2021)

  Changed paths:
    M flang/test/Evaluate/folding05.f90

  Log Message:
  -----------
  [flang] Switch from %f18 to %flang_fc1 in a test

This patch updates the final test that can be shared between the old and
the new Flang drivers and that has not been ported yet. %f18 (always
expanded as `f18`) is replaced with %flang_fc1 (expanded as either `f18`
or `flang-new -fc1`, depending on `FLANG_BUILD_NEW_DRIVER`).

This test should've been updated in https://reviews.llvm.org/D100309,
but I missed it then. That's because this test contains non-ascii
characters and `grep -I %f18` (as well as other grep-like tools) skips
it because it's interpreted as a data/binary file. In fact, it's just a
text file with non-ascii chars.

Since this is an obvious omission from D100309 (reviewed, accepted and
merged), I'm sending this without a review to reduce the noise on
Phabricator.




More information about the All-commits mailing list