[all-commits] [llvm/llvm-project] 4e413e: [InstCombine] Temporarily do not drop volatile sto...

Nikita Popov via All-commits all-commits at lists.llvm.org
Thu Sep 10 07:19:13 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 4e413e16216d0c94ada2171f3c59e0a85f4fa4b6
      https://github.com/llvm/llvm-project/commit/4e413e16216d0c94ada2171f3c59e0a85f4fa4b6
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-09-10 (Thu, 10 Sep 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
    M llvm/test/Transforms/InstCombine/volatile_store.ll

  Log Message:
  -----------
  [InstCombine] Temporarily do not drop volatile stores before unreachable

See discussion in D87149. Dropping volatile stores here is legal
per LLVM semantics, but causes issues for real code and may result
in a change to LLVM volatile semantics. Temporarily treat volatile
stores as "not guaranteed to transfer execution" in just this place,
until this issue has been resolved.




More information about the All-commits mailing list